Add GH16805 RNTupleProcessor regression test#22644
Conversation
matildamarjamaki
requested review from
bellenot,
jblomer and
silverweed
as code owners
vepadulano
left a comment
vepadulano
left a comment
There was a problem hiding this comment.
Thank you @matildamarjamaki for this contribution, I believe it's a good addition! I left a couple of minor remarks but the changes look already good.
Test Results 22 files 22 suites 3d 12h 10m 3s ⏱️ For more details on these failures, see this check. Results for commit a9d2dcc. ♻️ This comment has been updated with latest results. |
vepadulano
left a comment
vepadulano
left a comment
There was a problem hiding this comment.
Thanks for addressing the comments! Just two last minor suggestions from my side
enirolf
left a comment
enirolf
left a comment
There was a problem hiding this comment.
Very nice! I have one minor comment from my side, once that's addressed from my side it's good to go.
| "gh16805_rntuple_friend_0.root", | ||
| "gh16805_rntuple_friend_1.root", | ||
| "gh16805_rntuple_friend_2.root" |
There was a problem hiding this comment.
To ensure consistency between terms, I would be in favor to use only "join" when talking about RNTuple, instead of "friends".
4 participants
Summary
Adds a regression test for the GH16805 RNTupleProcessor use case.
The test creates several RNTuples corresponding to the original TTree setup ("stepzero", "stepone" and "topLevelFriend"), composes them using "RNTupleProcessor::CreateChain" and "RNTupleProcessor::CreateJoin" and validates the resulting values with "EXPECT_EQ".
The goal is to provide a simple regression test covering a TTree-friends-like workflow implemented with RNTupleProcessor.